Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kev ma/appeals 56226 #23343

Merged
merged 15 commits into from
Oct 25, 2024
Merged

Kev ma/appeals 56226 #23343

merged 15 commits into from
Oct 25, 2024

Conversation

Kevma50287
Copy link
Contributor

@Kevma50287 Kevma50287 commented Oct 24, 2024

Resolves https://jira.devops.va.gov/browse/APPEALS-56226

Description

Validated Motion to Advance On Docket task for inactive appeals is in dropdown during correspondence intake process and is generated in the backend task table and associated to the correspondence through correspondence_appeals table.

Added feature tests to submit_intake_form_spec.rb under the correspondence folder. Feature test iterates through all inactive appeals tasks, creates them through the frontend, and then performs checks to ensure that the task is created and assigned to the proper orgnanizations, and does not affect the inactive appeal's status

Acceptance Criteria

  • Code compiles correctly

Testing Plan

  1. Go to https://jira.devops.va.gov/browse/APPEALS-56226

Backend

Best practices

Code Documentation Updates

  • Add or update code comments at the top of the class, module, and/or component.

Tests

Test Coverage

Did you include any test coverage for your code? Check below:

  • RSpec
  • Jest
  • Other

Code Climate

Your code does not add any new code climate offenses? If so why?

  • No new code climate issues added

Kevma50287 and others added 4 commits October 23, 2024 17:32
* update restrictions to add task

* update check_inbound_ops_team_user method

* update specs

* update specs
@Kevma50287 Kevma50287 self-assigned this Oct 24, 2024
@Kevma50287 Kevma50287 marked this pull request as ready for review October 24, 2024 22:00
@@ -193,5 +193,21 @@ def inbound_ops_team_admin_setup
User.authenticate!(user: current_user)
end

def retry_if_not_found(max_attempts: 3, cleanup: nil)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@cacevesva cacevesva merged commit 3bf67fc into feature/APPEALS-39909 Oct 25, 2024
8 of 17 checks passed
@cacevesva cacevesva deleted the KevMa/APPEALS-56226 branch October 25, 2024 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants